Skip to content

Remove rapidsai from channel resolution to avoid rapids-xgboost - #221

Merged
rapids-bot[bot] merged 1 commit into
rapidsai:mainfrom
chyunsu3:xgboost_fix
Sep 10, 2026
Merged

Remove rapidsai from channel resolution to avoid rapids-xgboost#221
rapids-bot[bot] merged 1 commit into
rapidsai:mainfrom
chyunsu3:xgboost_fix

Conversation

@chyunsu3

@chyunsu3 chyunsu3 commented Sep 10, 2026 •
edited by csadorf
Loading

Copy link
Copy Markdown
Contributor

Unblock the CI by temporarily removing rapidsai from channel resolution. Right now, the CI is pulling an old version of XGBoost due to the recent removal of rapids-xgboost. See NVIDIA/cuml#8596 for the context.

Closes #222

@chyunsu3
chyunsu3 requested a review from a team as a code owner September 10, 2026 03:03
@chyunsu3
chyunsu3 requested a review from msarahan September 10, 2026 03:03
@github-actions github-actions Bot added the conda Relates to conda packaging label Sep 10, 2026
@chyunsu3 chyunsu3 added non-breaking Introduces a non-breaking change ci and removed conda Relates to conda packaging labels Sep 10, 2026
@coderabbitai

coderabbitai Bot commented Sep 10, 2026 •
edited
Loading

Copy link
Copy Markdown

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 5f9c69f8-e2d1-44a7-bae5-cc2909273d98

📥 Commits

Reviewing files that changed from the base of the PR and between b372dbc and c2ebde8.

📒 Files selected for processing (9)
  • conda/environments/all_cuda-129_arch-aarch64.yaml
  • conda/environments/all_cuda-129_arch-x86_64.yaml
  • conda/environments/all_cuda-133_arch-aarch64.yaml
  • conda/environments/all_cuda-133_arch-x86_64.yaml
  • conda/environments/clang_tidy_cuda-129_arch-x86_64.yaml
  • conda/environments/clang_tidy_cuda-133_arch-x86_64.yaml
  • conda/environments/cpp_all_cuda-129_arch-x86_64.yaml
  • conda/environments/cpp_all_cuda-133_arch-x86_64.yaml
  • dependencies.yaml
💤 Files with no reviewable changes (9)
  • conda/environments/cpp_all_cuda-133_arch-x86_64.yaml
  • conda/environments/clang_tidy_cuda-133_arch-x86_64.yaml
  • conda/environments/cpp_all_cuda-129_arch-x86_64.yaml
  • dependencies.yaml
  • conda/environments/clang_tidy_cuda-129_arch-x86_64.yaml
  • conda/environments/all_cuda-129_arch-aarch64.yaml
  • conda/environments/all_cuda-129_arch-x86_64.yaml
  • conda/environments/all_cuda-133_arch-aarch64.yaml
  • conda/environments/all_cuda-133_arch-x86_64.yaml

Included review availability: Your plan provides up to 12 included reviews per hour; 11 remain after this review.

📝 Summary

Summary by CodeRabbit

  • Chores
    • Updated Conda environment configurations to use rapidsai-nightly instead of rapidsai.
    • Retained conda-forge where previously configured.
    • Applied the channel updates across CUDA 12.9 and 13.3 environments, including ARM64 and x86_64 configurations.

Walkthrough

Changes

Conda channel update

Layer / File(s) Summary
Channel source configuration
dependencies.yaml		 The shared Conda channel entry changes from rapidsai to rapidsai-nightly.
Environment configurations
conda/environments/*		 CUDA and tooling environment files remove or replace the rapidsai channel with rapidsai-nightly.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to c2ebd

This update removes rapidsai from channel resolution to avoid obsolete XGBoost selection; no concrete merge-blocking risk is established by the supplied evidence.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly and concisely describes the main change: removing the rapidsai channel to prevent rapids-xgboost resolution.
Description check ✅ Passed The description explains the CI issue, the temporary channel removal, and the connection to the recent rapids-xgboost removal. It is directly related to the changeset.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands.

@chyunsu3 chyunsu3 added the bug Something isn't working label Sep 10, 2026
@chyunsu3 chyunsu3 mentioned this pull request Sep 10, 2026
Open
@gforsyth

gforsyth commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Same as with NVIDIA/cuml#8596 we'll have to make sure to revert this change before cutting the release (reverting this shouldn't cause regressions once we've also removed the rapids-xgboost packages from rapidsai channel)

@chyunsu3

chyunsu3 commented Sep 10, 2026

Copy link
Copy Markdown
Contributor Author

/merge

@rapids-bot
rapids-bot Bot merged commit a7dd87d into rapidsai:main Sep 10, 2026
70 checks passed
@chyunsu3
chyunsu3 deleted the xgboost_fix branch September 10, 2026 13:34
@csadorf csadorf mentioned this pull request Sep 11, 2026
Open
2 tasks
@csadorf

csadorf commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

we'll have to make sure to revert this change before cutting the release

Created #223 to track the required revert before the 26.10 release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gforsyth gforsyth gforsyth approved these changes
@msarahan msarahan Awaiting requested review from msarahan msarahan is a code owner automatically assigned from rapidsai/packaging-codeowners

Assignees

No one assigned

Labels

bug Something isn't working ci non-breaking Introduces a non-breaking change

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[CI] Conda Python test environments fail to solve with XGBoost and RMM

3 participants

@chyunsu3 @gforsyth @csadorf